Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUILD] add loongarch info #3052

Merged
merged 4 commits into from
Sep 18, 2024
Merged

[BUILD] add loongarch info #3052

merged 4 commits into from
Sep 18, 2024

Conversation

loong-hy
Copy link
Contributor

@loong-hy loong-hy commented Sep 5, 2024

about loongarch: https://docs.kernel.org/arch/loongarch/introduction.html

`
......
Start 413: exporter.OstreamLogExporter.DefaultLogRecordToCout
413/421 Test #413: exporter.OstreamLogExporter.DefaultLogRecordToCout ................................................... Passed 0.00 sec
Start 414: exporter.OStreamLogRecordExporter.SimpleLogToCout
414/421 Test #414: exporter.OStreamLogRecordExporter.SimpleLogToCout .................................................... Passed 0.00 sec
Start 415: exporter.OStreamLogRecordExporter.LogWithStringAttributesToCerr
415/421 Test #415: exporter.OStreamLogRecordExporter.LogWithStringAttributesToCerr ...................................... Passed 0.00 sec
Start 416: exporter.OStreamLogRecordExporter.LogWithVariantTypesToClog
416/421 Test #416: exporter.OStreamLogRecordExporter.LogWithVariantTypesToClog .......................................... Passed 0.00 sec
Start 417: exporter.OStreamLogRecordExporter.IntegrationTest
417/421 Test #417: exporter.OStreamLogRecordExporter.IntegrationTest .................................................... Passed 0.00 sec
Start 418: exporter.OStreamLogRecordExporter.IntegrationTestWithEventId
418/421 Test #418: exporter.OStreamLogRecordExporter.IntegrationTestWithEventId ......................................... Passed 0.00 sec
Start 419: exporter.OStreamLogRecordExporter.Factory
419/421 Test #419: exporter.OStreamLogRecordExporter.Factory ............................................................ Passed 0.00 sec
Start 420: exporter.InMemorySpanData.AddRecordable
420/421 Test #420: exporter.InMemorySpanData.AddRecordable .............................................................. Passed 0.00 sec
Start 421: exporter.InMemorySpanExporter.ExportBatch
421/421 Test #421: exporter.InMemorySpanExporter.ExportBatch ............................................................ Passed 0.00 sec

100% tests passed, 0 tests failed out of 421

Total Test time (real) = 11.78 sec

`

@loong-hy loong-hy requested a review from a team September 5, 2024 08:30
Copy link

linux-foundation-easycla bot commented Sep 5, 2024

CLA Signed


The committers listed above are authorized under a signed CLA.

@marcalff marcalff changed the title add loongarch info [BUILD] add loongarch info Sep 5, 2024
@marcalff
Copy link
Member

marcalff commented Sep 5, 2024

@loong-hy

Thanks for the patch.

Please sign the CLA so it can be merged.

@marcalff marcalff added the pr:waiting-on-cla Waiting on CLA label Sep 5, 2024
Copy link

codecov bot commented Sep 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.78%. Comparing base (497eaf4) to head (9035790).
Report is 131 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3052      +/-   ##
==========================================
+ Coverage   87.12%   87.78%   +0.66%     
==========================================
  Files         200      195       -5     
  Lines        6109     5953     -156     
==========================================
- Hits         5322     5225      -97     
+ Misses        787      728      -59     

see 125 files with indirect coverage changes

@lalitb
Copy link
Member

lalitb commented Sep 6, 2024

LGTM once the CLA is signed.

Copy link
Member

@marcalff marcalff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM once the CLA is signed.

@ThomsonTan
Copy link
Contributor

@loong-hy thanks for the contribution. Could you please take look at signing the CLA so we can merge the PR?

@marcalff
Copy link
Member

/easycla

@loong-hy
Copy link
Contributor Author

@loong-hy thanks for the contribution. Could you please take look at signing the CLA so we can merge the PR?

Already signed. Thank you.

@marcalff marcalff merged commit 2314e06 into open-telemetry:main Sep 18, 2024
52 checks passed
@marcalff marcalff removed the pr:waiting-on-cla Waiting on CLA label Sep 18, 2024
malkia added a commit to malkia/opentelemetry-cpp that referenced this pull request Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants